Skip to content

Fixed the link - #48

Merged
alkohli merged 1 commit into
Azure-Samples:mainfrom
alkohli:foup
Apr 29, 2025
Merged

Fixed the link#48
alkohli merged 1 commit into
Azure-Samples:mainfrom
alkohli:foup

Conversation

@alkohli

@alkohli alkohli commented Apr 29, 2025

Copy link
Copy Markdown
Contributor

Purpose

  • ...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@alkohli
alkohli requested a review from ManikaDhiman April 29, 2025 13:32
ronmiab
ronmiab reviewed Apr 29, 2025
View reviewed changes

@ronmiab ronmiab left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alkohli - your change looks good me.

@alkohli
alkohli removed the request for review from ManikaDhiman April 29, 2025 14:28
@alkohli
alkohli merged commit 42fb342 into Azure-Samples:main Apr 29, 2025
ManikaDhiman
ManikaDhiman reviewed Apr 29, 2025
View reviewed changes

@ManikaDhiman ManikaDhiman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ronmiab ronmiab ronmiab approved these changes
+1 more reviewer
@ManikaDhiman ManikaDhiman ManikaDhiman left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alkohli @ManikaDhiman @ronmiab